This is broken for me. If I run ./run_scraper_data.sh sonoma or python scraper_data.py sonoma, I get:

Traceback (most recent call last):
  File "scraper_data.py", line 4, in <module>
    from covid19_sfbayarea import data as data_scrapers
  File "/Users/rbrackett/Dev/sfbrigade/data-covid19-sfbayarea/covid19_sfbayarea/data/__init__.py", line 4, in <module>
    from . import sonoma
  File "/Users/rbrackett/Dev/sfbrigade/data-covid19-sfbayarea/covid19_sfbayarea/data/sonoma.py", line 8, in <module>
    from format_error import FormatError
ModuleNotFoundError: No module named 'format_error'

Does it work for you? Seems like it should be:

(It also shouldn’t need need the # type: ignore comment.)

Also, while you’re here, maybe it’s worthwhile to make this a shared module up in covid19_sfbayarea/errors.py instead of covid19_sfbayarea/data/format_error.py? Some of the news scrapers also use an identical class, and we should really only have one implementation. Then this line would be:

Minor nit: one of the important things about naming (or aliasing) types like this is to change how you conceptualize your values and functions (e.g. you shouldn’t be thinking of UnformattedSeriesItem like a shortcut for Dict[str, str] here; you should be thinking of it like a subclass of dict — it should conceptually be its own separate thing).

So if you’re changing the return type to something named UnformattedSeriesItem, it’s probably a good idea to change the function name to not talk about making a dict and instead call it something like row_list_to_series_item or something.

Oh, forgot to comment on this in the review: I don’t think there’s any reason to convert this to a list, since you’re only iterating over it once and not returning it: